Skip to content

Conversation

bchetioui
Copy link
Member

The linter messed up the order of includes, which is necessary as is.

@llvmbot llvmbot added mlir mlir:bufferization Bufferization infrastructure labels Apr 16, 2025
@llvmbot
Copy link
Member

llvmbot commented Apr 16, 2025

@llvm/pr-subscribers-mlir-bufferization

@llvm/pr-subscribers-mlir

Author: Benjamin Chetioui (bchetioui)

Changes

The linter messed up the order of includes, which is necessary as is.


Full diff: https://github.com/llvm/llvm-project/pull/135962.diff

1 Files Affected:

  • (modified) mlir/include/mlir/Dialect/Bufferization/IR/BufferizationTypeInterfaces.h (+1-1)
diff --git a/mlir/include/mlir/Dialect/Bufferization/IR/BufferizationTypeInterfaces.h b/mlir/include/mlir/Dialect/Bufferization/IR/BufferizationTypeInterfaces.h
index 8672aa60a43c5..af2cad006e176 100644
--- a/mlir/include/mlir/Dialect/Bufferization/IR/BufferizationTypeInterfaces.h
+++ b/mlir/include/mlir/Dialect/Bufferization/IR/BufferizationTypeInterfaces.h
@@ -13,7 +13,7 @@
 // Bufferization Type Interfaces
 //===----------------------------------------------------------------------===//
 
+#include "mlir/IR/Types.h"  // NOLINT
 #include "mlir/Dialect/Bufferization/IR/BufferizationTypeInterfaces.h.inc"
-#include "mlir/IR/Types.h"
 
 #endif // MLIR_DIALECT_BUFFERIZATION_IR_BUFFERIZATIONTYPEINTERFACES_H_

Copy link

github-actions bot commented Apr 16, 2025

✅ With the latest revision this PR passed the C/C++ code formatter.

The linter messed up the order of includes, which is necessary as is.
@bchetioui bchetioui merged commit 9483aaa into llvm:main Apr 16, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mlir:bufferization Bufferization infrastructure mlir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants